Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rollup of 10 pull requests #47962

Merged
merged 20 commits into from
Feb 3, 2018
Merged

Rollup of 10 pull requests #47962

merged 20 commits into from
Feb 3, 2018

Conversation

SimonSapin and others added 10 commits November 21, 2017 14:44
is_unsafe_place only filters out statics in the rhs, not the lhs. Since
it's possible to reach that 'Place::Static', we handle statics the same
way as we do locals.

Fixes rust-lang#47789
Currently ', " and \ are escaped as \', \" and \\ respectively. This
leads to confusing messages such as `error: unknown start of token: \\`
when encountering a single backslash.

Fix by emitting printable ASCII characters directly. This will still
escape \r, \n, \t and Unicode characters.

Fixes rust-lang#47902
The `exchange_free` lang item is gone in favour of `box_free` [1].

Some warnings are also fixed by this commit.

[1]: rust-lang@ca115dd
The current `f32|f64.to_degrees` implementation uses a division to calculate 180/π, which causes a loss of precision. Using a constant is still not perfect (implementing a maximally-precise algorithm would come with a high performance cost), but improves precision with a minimal change.
Sometimes the parser attempts to synthesize spans from within a macro
context with the span for the captured argument, leading to non-sensical
spans with very bad output. Given that an incorrect span is worse than
a partially incomplete span, when detecting this situation return only
one of the spans without mergin them.
@rust-highfive
Copy link
Collaborator

Some changes occurred in HTML/CSS.

cc @GuillaumeGomez

@rust-highfive
Copy link
Collaborator

r? @steveklabnik

(rust_highfive has picked a reviewer for you, use r? to override)

@kennytm
Copy link
Member Author

kennytm commented Feb 2, 2018

@bors r+ p=6

@bors
Copy link
Contributor

bors commented Feb 2, 2018

📌 Commit 5b086f4 has been approved by kennytm

@kennytm kennytm added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Feb 2, 2018
@bors
Copy link
Contributor

bors commented Feb 2, 2018

⌛ Testing commit 5b086f4f9ac5a751f6cce715cd68ef26c599cf05 with merge a14aee5e646df76254a585ff56172d9072746090...

@bors
Copy link
Contributor

bors commented Feb 2, 2018

💔 Test failed - status-travis

@kennytm
Copy link
Member Author

kennytm commented Feb 2, 2018

Legit, caused by #47753.

[00:13:51] invalid license MPL-2.0 in /checkout/obj/build/tmp/distcheck/src/vendor/toml-query/Cargo.toml
[00:13:51] invalid license MPL-2.0 in /checkout/obj/build/tmp/distcheck/src/vendor/is-match/Cargo.toml

…uggest removing value from `break` when invalid When attempting to use `break` with a value in a type of loop where it'd be invalid (any non-`loop`), suggest using `break` on its own. Close rust-lang#34359.
…ove dead code The Clone impl makes the lint ignore the type.
…sakis Fix ICE when assigning references to a static mut with NLL is_unsafe_place only filters out statics in the rhs, not the lhs. Since it's possible to reach that 'Place::Static', we handle statics the same way as we do locals. Fixes rust-lang#47789
…r-msg, r=petrochenkov Improve char escaping in lexer messages Currently ', " and \ are escaped as \', \" and \\ respectively. This leads to confusing messages such as `error: unknown start of token: \\` when encountering a single backslash. Fix by emitting printable ASCII characters directly. This will still escape \r, \n, \t and Unicode characters. Fixes rust-lang#47902
…ms box example code The `exchange_free` lang item is gone in favour of `box_free` [1]. Some warnings are also fixed by this commit. [1]: rust-lang@ca115dd
…ppe Use constant for 180/π in to_degrees The current `f32|f64.to_degrees` implementation uses a division to calculate `180/π`, which causes a loss of precision. Using a constant is still not perfect (implementing a maximally-precise algorithm would come with a high performance cost), but improves precision with a minimal change. As per the discussion in rust-lang#29944, this fixes rust-lang#29944 (the costs of improving the precision further would not outweigh the gains).
…s Minimize weird spans involving macro context Sometimes the parser attempts to synthesize spans from within a macro context with the span for the captured argument, leading to non-sensical spans with very bad output. Given that an incorrect span is worse than a partially incomplete span, when detecting this situation return only one of the spans without merging them. Fix rust-lang#32072, rust-lang#47778. CC rust-lang#23480.
…etMisdreavus Fix ugly hover in sidebar In the sidebar, the elements under `Structs`, `Enums`... have an ugly hover if they're not selected. This fixes it. r? @QuietMisdreavus
@kennytm
Copy link
Member Author

kennytm commented Feb 2, 2018

@bors r+

Removed #47753

@bors
Copy link
Contributor

bors commented Feb 2, 2018

📌 Commit 8d1586d has been approved by kennytm

@kennytm kennytm changed the title Rollup of 10 pull requests Rollup of 9 pull requests Feb 2, 2018
@bors
Copy link
Contributor

bors commented Feb 2, 2018

⌛ Testing commit 8d1586d with merge 15dd8c39c0cc3b88b4e2cb2fb2eec1311d61e027...

The comment referred to a variable using an incorrect name. (it has probably been renamed since the comment was written, or the comment was copied elsewhere - I noted the example in libcore has the `tmp` name for the temporary variable.)
@bors
Copy link
Contributor

bors commented Feb 2, 2018

💔 Test failed - status-appveyor

@kennytm kennytm changed the title Rollup of 9 pull requests Rollup of 10 pull requests Feb 3, 2018
copy_nonoverlapping example: Fixed typo

The comment referred to a variable using an incorrect name. (it has probably been renamed since the comment was written, or the comment was copied elsewhere - I noted the example in libcore has the `tmp` name for the temporary variable.)
@kennytm
Copy link
Member Author

kennytm commented Feb 3, 2018

@bors r+

Added #47973.

@bors
Copy link
Contributor

bors commented Feb 3, 2018

📌 Commit 3a0a423 has been approved by kennytm

@bors
Copy link
Contributor

bors commented Feb 3, 2018

⌛ Testing commit 3a0a423 with merge 8d04b8f...

bors added a commit that referenced this pull request Feb 3, 2018
Rollup of 10 pull requests

- Successful merges: #46156, #47829, #47842, #47898, #47914, #47916, #47919, #47942, #47951, #47973
- Failed merges: #47753
@bors
Copy link
Contributor

bors commented Feb 3, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: kennytm
Pushing 8d04b8f to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.